testsuite/gtk/defaultvalue: Actually build...& fix

It looks like this got dropped during the move from autotools and never
restored. I can see why, since making it work wasn't a hugely fun task!

Notes on some less then obvious details:
 * PlacesSidebar is private now and didn't seem to be to be particularly
   easy to adapt to, so this moves to checking for it by name, not TYPE.
   I couldn't find a (fast) better way; if you know how, please clean up
 * added 2 casts to avoid warnings from the new type-propagating ref()
 * GdkClipboard and GdkContentProvider need some properties dodged
 * GtkToolItemGroup is gone
 * fixed indentation and used TypeName:property-name syntax in a print()

Merge branch 'wip/cursor-output-scale-changes' into 'master'

wayland: Update cursor surface scale when output scale changes

See merge request GNOME/gtk!224

Merge branch 'wip/aplazas/modelbutton-markup-master' into 'master'

ModelButton: Add 'use-markup' property

See merge request GNOME/gtk!222

wayland: Update cursor surface scale when output scale changes

Also update the cursor surfaces of every seat when an output changes
scale. This could for example happen when a monitor scale is changed via
Settings.

widget: Move opactiy == 0 check

If we check it too early, we will not unset priv->draw_neeeded, which
will then cause queue_draw() calls to not have an effect later. And that
causes changes in opacity to not register.

Closes #1180

notify test: Remove more nonexistent properties

notify test: Remove some non-existant properties

testsuite: Remove a GtkColorButton special case

The comment above explains neatly why subclassing GtkButton for
GtkColorButton was a bad idea. Nowadays it's a GtkWidget subclass
containing a GtkButton so let's remove the special case here.

Remove some GtkRecentChooserMenu references

Doesn't exist anymore.

ModelButton: Add 'use-markup' property

Binds this property to the button's label, allowing a model button to
have text with markup.

This will be convenient for buttons like 'Online Accounts ^↗'.

gdk: Fix some g-i annotations warnings

gl renderer: Ignore 0-sized fallback nodes

Do not modify a const GdkEvent in place

When deciding whether or not to emulate a press event, we're translating
the last event coordinates and mutating the given event structure
unconditionally.

We should modify the newly created GdkEvent copy, since it's what we're
going to use when emitting the press event.

This avoids mutating a constant GdkEvent and global state, and also
avoids a compiler warning.

Docs: Remove more since markers

The idea is that GTK+ 4 will be an epoch, API-wise.
Everything that was around for 4.0 has been there
since the beginning of the epoch and doesn't need
markers.

docs: Document GtkEventControllerKey

The long description was missing.

Bump version to 3.94.0

Remove a lot of Since annotations

4.0 will represent a clean epoch. We don't want to have
lots of noise in the docs about 2.x or 3.x.

Some updates

reftests: Fix compilation

FOREIGN surfaces are gone.

gdk: Remove GDK_SURFACE_FOREIGN

We don't create surfaces of that type anymore.

imwayland: Fix a small leak

If the parent get_preedit_string implementation returns a nonnull
zero-length string, then we ignore it, which is almost fine. We have to
free it, though.

Fixes #1174

Merge branch 'master' into 'master'

gtk: also recolor circle and ellipse in symbolic SVG icons.

See merge request GNOME/gtk!214

gtk: also recolor circle and ellipse in symbolic SVG icons.

button: Remove unused variable

scrolledwindow: Avoid a queue_resize path in size-allocate

gtktreeviewcolumn: Implement column dragging through GtkGestureDrag

And remove ::event signal handler.

gtktexthandle: Update to gtk4 rendering/input

GtkTextHandle was neglected by whoever removed the ::draw signal,
leaving it entirely broken. Update to using GtkGizmo so we can
implement snapshot of text handles.

Input has received a revamp too, handling is done through a
GtkGestureDrag and coordinate calculations simplified by storing
the delta to the hotspot on ::begin instead of ::update, as this
value is constant throughout the gesture. Widget state management
on crossing events happens implicitly, so no longer needs to be
done here.

Last but not least, CSS has also been updated so handles are
rendered at the correct size and proportion, and with the padding
that code expects of it.

gtkwindow: Queue popover resize on set_popover_position()

So the new position is eventually honored. Fixes magnifier and handles
staying in place after being shown.

gtkfilechooserwidget: Forward events to filechooser through key controller

Instead of manually calling gtk_binding_set_activate_event() on an ::event
callback in the save entry.

gtkfilechoooserentry: Handle tab completion through key controller

gtkemojicompletion: Use GtkEntry key controller to handle key presses

gtkentrycompletion: Properly remove controller from entry

When disconnecting the GtkEntry from the GtkEntryCompletion, we
must remove the controller in order to avoid dangling callbacks.

gtkentrycompletion: Move focus out handling to key controller

We can use the already present key controller to handle focus out.

gtkentrycompletion: Perform event handling on popup through controllers

Instead of a ::event signal handler.

gtkplacessidebar: Replace ::event signal handlers with gestures

Set up a gesture on the sidebar rows to detect pointer clicks on
it. The row DnD management has been moved to the row widget itself,
it makes more sense even if the drag is began from the sidebar widget.

gtkwindow: Move map/delete/configure event management to gtkmain

Toplevels are about the only widgets interested in these events,
just handle those without going through the capture/bubble handling
code.

gtkwindow: Simplify WM drag/resize handling

We still need a drag gesture both on front (capture) and back (bubble)
to handle dragging from both the GtkWindow widget and chrome in the
headerbar. But we can do it through 2 drag gestures, instead of special
event handling code.

gtkmenushell: Port to GtkGesture

We still need to poke the current event at places, but this is
better than the ::event vfunc.

gtkpopover: Fix key navigation

This has been broken since we switched key event delivery to follow
the same semantics than pointer/touch. There, GTK+ grabs will influence
the topmost widget during event delivery, rendering the toplevel
unable to handle key navigation. The toplevel must handle those key
events in an explicit manner then.

We don't render the keyboard focus rectangle yet, but I assume that's
something else.

gtkpopover: Port to GtkEventController/GtkGesture

Use GtkEventControllerKey and GtkGestureMultiPress to replace key/button
event handling.

gtkwindow: Handle focus in/out through key controller

One less use of ::event in this widget, now mostly left to
map/delete/configure, those puny events.

gtkspinbutton: Use GtkEventControllerKey

And drop the ::event vfunc implementation with it.

gtkfontchooser: Use GtkEventControllerKey

Use an event controller on GtkFontChooserDialog, a nice side effect
is that we can use gtk_event_controller_key_forward() and
gtk_search_entry_set_key_capture_widget() instead of passing events
around for dialog search.

gtkfilechooserwidget: Drop usage of ::event vmethod

Use a key controller set up in the UI file instead.

gtkfilechooserentry: Use GtkEntry key controller for focus-out handling

Expose the GtkEntry key controller in private API, so we don't have to
create yet another one just to handle focus-out.

gtkentry: Use gestures for entry icons

Instead of doing all handling manually in the ::event vfunc,
set up drag/multipress gestures on icon images, and implement
emission of ::icon-press/release and DnD there.

As a side effect, the GdkEvent field in ::icon-press/release
signals has been dropped. Callers that might be interested on it
may still use gtk_get_current_event*().

gtkbutton: remove keyboard device GTK+ grab

This isn't really necessary, if keyboard focus forcibly goes somewhere
else we will get ::grab-notify, which is sufficient to deactivate the
button again.

gtkbutton: Use key controller for button (de)activation through keybindings

And stop using GtkWidget::event for good.

x11: Fix dnd action handling

We were mistakenly assigning an Atom to a flags value.

EmojiCompletion: Avoid another un/signed warning

The cherry-pick missed this, I guess.

EmojiCompletion: Avoid un/signed compare warnings

(A) Use gsize to match the result of g_variant_n_children
(B) Use guint for n_matches, like the struct (and all other n_matches)

themes: Remove leftover bit of .linked workaround

Now that .linked acts like the .path-bar already did, there is no point
in applying the same styles again (with higher specificity).

container: Remove SPECIAL_CONTAINER hack

As discussed in !129

Closes !129

stackswitcher: Remove icon-size property

This ought to be controlled via css these days.

infobar: Remove priv pointer

Remove some unnecessary snapshot implementations

fixed: Remove priv pointer

listbox: Use widget insert API instead of fiddling with CSS nodes

button: Remove ICON_SIZE_BUTTON mention in docs

The icon will have the size given by css, not whatever
GTK_ICON_SIZE_BUTTON was.

adjustment: Only emit ::changed from dispatch_properties_changed

adjustment: Remove adjustment_changed_stamp

testsuite: Print tested property name in notify test

adjustment: Use public setters in set_property

themes: Regenerate CSS for recent commits

e.g. the box.linked one, which didn't update the compiled CSS.

HC: Avoid excessive selectors from prev commit

The new rule does not need to affect all those other pseudoclasses. I
just put it in the wrong scope.

HC: Avoid same BG/FG colors in flat treeview entry

Selected rows in tree views in HighContrast have a background colour the
same or nearly as the normal text colour, so we cannot let entries in
such rows have transparent backgrounds, or the text inside the entry
becomes nearly or totally impossible to see.

Dodge this by giving entry.flat inside treeview and with :focus the
$base_color, which is different from the text & so lets that be seen.

https://gitlab.gnome.org/GNOME/gtk/merge_requests/125

searchenginetracker: Don't leak the hits' GFiles

https://gitlab.gnome.org/GNOME/gtk/merge_requests/206

Merge branch 'issue1084' into 'master'

application: Complete the startup sequence before quitting remote instances

Closes #979

See merge request GNOME/gtk!156

Merge branch 'master' into 'issue1084'

# Conflicts:
#   gdk/gdk.c

window: Remove priv pointer

pathbar: Remove unused define

docs: Remove a duplicate line in gdk4-sections.txt

This was causing another docs build break.

docs: fix a typo

Invalid syntax in gdk4-sections.txt was causing docs builds to fail.

Merge branch '1160-crash-clearing-gtkentry-icon-tooltip-master' into 'master'

Resolve "Crash clearing GtkEntry icon tooltip"

Closes #1160

See merge request GNOME/gtk!205

win32: Fix typo

This should fix gtk-doc erroring out.

Revert "Add aligned allocator functions to GSK"

This reverts commit 8e74eb382f0c617b7b65a6850952d6acc9fd3018.

This code is not necessary. It worked around a bug in graphene where
graphene was requiring stricter alignment than glib allocators could
guarantee.

Revert "Use aligned allocators for GtkSnapshot"

This reverts commit c02bc22cc575470aab870599f2781c3781fb75e6.

This code is not necessary.
The bug causing this problem ws prsent in the graphene library.

rendernodepaintable: Fix rendering position

When the given size had an offset, we were moving that offset in the
wrong direction.

dnd: Use a GtkPicture in gtk_dnd_set_icon_paintable()

We don't want to get the paintable scaled down to icon size.

Also set can-shrink = FALSE to replicate GTK3 behavior with size
requests.

snapshot: Compute correct size in gtk_snapshot_to_paintable()

Also, do actually respect the passed in size argument if it isn't NULL.

Fixes text being cut off inside DND icons.

dnd: gdk_drag_action_is_unique() should return a boolean

a11y/entry: Fix copy-pasteo re 2ndary icon tooltip

The else case was wrongly resetting the accessible description on the
primary icon, which might not exist and can therefore cause a crash.

https://gitlab.gnome.org/GNOME/gtk/issues/1160

Merge branch 'wip/carlosg/issue-1159' into 'master'

Fix issue 1159

Closes #1159

See merge request GNOME/gtk!202

Update POTFILES.in

wayland: Use shm_open(SHM_ANON) on FreeBSD

This functionality is similar to Linux's memfd. It creates anonymous shared memory without touching the filesystem, which allows it to work in Capsicum capability mode (sandbox).

gtkgesture: Check claimed status after event delivery

The claimed status check should happen after ::end is emitted,
as the gesture may deny the sequence that much late. In this
case the event should keep propagating.

https://gitlab.gnome.org/GNOME/gtk/issues/1159

Closes: #1159

gtkwidget: Avoid doubly coordinate conversion when emulating press

We are poking again into the event propagation machinery, which
expects events in toplevel coordinates. Since we can't fetch the
original event back at this point, translate the coordinates
back to the toplevel so the emulated press ends up in the right
place.

https://gitlab.gnome.org/GNOME/gtk/issues/1159

Closes: #1159

x11: Set background as transparent as possible

Instead of making the background black, make it transparent black for
RGBA visuals and set a None background otherwise.

widget: Remove time argument from drag_data_get() vfunc

dnd: Remove unused time arguments

dnd: Remove GdkDragContext.dest_surface

The variable is now unused

GDK W32: Remove gdk_win32_surface_foreign_new_for_display

That function is now unused.

GDK W32: Adapt to dest_surface removal, misc changes

* There's no GdkDragContext->dest_surface anymore.
  Add dest_window field to GdkWin32DragContext,
  and use that instead.
* Remove unused function prototypes
* Add more comments
* Rename variables and fields from 'window' to 'surface'
  where appropriate
* Fix header indentation a bit
* Try to ensure that uninitialized/unknown handle variables
  and fields are set to INVALID_HANDLE_VALUE instead of NULL,
  as there may be cases where NULL is a valid handle value.

dnd: Remove GdkDragContext.is_source

All drag contexts are sources these days, the other ones are GdkDrop
now.

x11: Remove gdk_x11_surface_foreign_new_for_display()

People who want to use foreign windows should use X directly.

x11: Store the XID instead of a GdkSurface

This might be foreign Windows and we don't want to create surfaces for
those.

Also, stop using GdkDragContext.dest_surface, that variable is meant to
go away.

x11: Simplify test

Now that we have gdk_drag_context_get_display(), use it.

x11: xdnd_send_event() always returns TRUE

Make it return nothing instead and delete the code that handled the
FALSE case.

x11: Don't multiply X values by the scale factor

This is fallout from a too eager change in
bdb442be21471a91820219347b1345ed7b84d034.

dnd: Make the base Drag/Drop types abstract

This way, they'll be ignored by the testsuite.

And we don't want to instantiate them anyway, the backends have
their own subclasses.